fix npm cache clean - #420

uh, how? is that file generated?

It is locally generated before pushing, the template lives in the root folder: Dockerfile-onbuild.template

I'd argue tho that this only makes sense for node 8

The same template generates all dockerfiles, so the Node 8 one will be overwritten in future releases unless the template is updated. The --force param is backwards-compatible.

I'll put it in the template then 👍

Could you rebase your branch on master?

git fetch origin/maseter
git rebase --no-ff origin/master

And then you'll need to force push to your forked branch.

I'm confused, I can't see a change to the npm cache clean line. Am I am I missing something?

@gibfahn that links points to master, this pr hasn't been merged yet :)

@Starefossen rebased 🙌

@itajaja the PR is still broken, closing. I've pushed #426 which is a proper fix.

sorry, don't know what happened, I just rebased in a rush and I probably messed something